Skip to content

Conversation

nirLumigo
Copy link
Contributor

Before:
image

After:
image

def is_supported(event) -> bool:
return safe_get(event, ["Records", 0, "EventSource"]) == "aws:sns"
def is_supported(event: Event) -> bool:
return safe_get(event.raw_event, ["Records", 0, "EventSource"]) == "aws:sns"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use record_event_source

@codecov
Copy link

codecov bot commented Feb 14, 2021

Codecov Report

Merging #180 (2219b6f) into master (827a7a9) will increase coverage by 1.26%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
+ Coverage   96.21%   97.48%   +1.26%     
==========================================
  Files          11       22      +11     
  Lines         952     1786     +834     
==========================================
+ Hits          916     1741     +825     
- Misses         36       45       +9     
Impacted Files Coverage Δ
lumigo_tracer/sync_http/sync_hook.py
lumigo_tracer/parsers/http_data_classes.py
lumigo_tracer/sync_http/handler.py
lumigo_tracer/parsers/utils.py
lumigo_tracer/parsers/event_parser.py
lumigo_tracer/utils.py
lumigo_tracer/parsers/parser.py
lumigo_tracer/wrappers/sql/sqlalchemy_wrapper.py 89.74% <0.00%> (ø)
lumigo_tracer/wrappers/http/sync_http_wrappers.py 95.17% <0.00%> (ø)
lumigo_tracer/wrappers/http/http_parser.py 99.32% <0.00%> (ø)
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2493b8...2219b6f. Read the comment docs.

@nirLumigo nirLumigo merged commit 232dd85 into master Feb 15, 2021
@nirLumigo nirLumigo deleted the RD-4609-unmask-ddb-key branch February 15, 2021 07:22
saartochner-lumigo pushed a commit that referenced this pull request Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants